Skip to content

updating re_show_redundancy to check for null value#344

Open
cheenpo wants to merge 1 commit into
networktocode:developfrom
cheenpo:fix_redundancy
Open

updating re_show_redundancy to check for null value#344
cheenpo wants to merge 1 commit into
networktocode:developfrom
cheenpo:fix_redundancy

Conversation

@cheenpo

@cheenpo cheenpo commented Feb 17, 2026

Copy link
Copy Markdown

NTC-5197

@cheenpo

cheenpo commented Feb 18, 2026

Copy link
Copy Markdown
Author

copying in feedback where this original commit was patched:

We did attempt your fix, and that did work to allow the file transfers to be conducted on routers.
That being said, we also noticed that the cause of the errors were caused by console logs bleeding through in the command output causing the re.match to anchor on those logs, so, we switched re.match to re.search and also rectified the issue 

@jeffkala

Copy link
Copy Markdown
Contributor

Needs unittest

@gsnider2195

Copy link
Copy Markdown
Contributor

we also noticed that the cause of the errors were caused by console logs bleeding through in the command output

It sounds like logging synchronous may be turned off?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants